Skip to content

fix(deps): update Azure Identity to 1.18.4#779

Open
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-azure-identity
Open

fix(deps): update Azure Identity to 1.18.4#779
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-azure-identity

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

  • update the example-only com.azure:azure-identity dependency from 1.15.0 to the latest stable release, 1.18.4
  • pick up Azure's supported transitive stack: Netty 4.1.135.Final and Reactor Netty 1.2.18
  • remove the vulnerable Nimbus JOSE JWT and json-smart path that was present through the older MSAL dependency graph

This is expected to close 33 open Dependabot alerts after GitHub rescans the default branch: 36, 37, 38, 47, 48, 50, 51, 53, 55, 61, 62, 68, and 70 through 90.

Compatibility

This dependency is declared only by openai-java-example, which does not apply openai.publish and has no publishing tasks. It is not present in the POM or runtime graph of any published OpenAI Java artifact, so this update cannot change downstream consumers' dependency resolution.

The example uses only AuthenticationUtil.getBearerTokenSupplier(TokenCredential, String...) and new DefaultAzureCredentialBuilder().build(). Those public binary signatures are identical in 1.15.0 and 1.18.4, the example compiles unchanged, and both Azure Identity artifacts target Java 8 bytecode.

The upstream 1.17.0 release removed the deprecated SharedTokenCacheCredential from the default credential chain. That can affect a developer running this source-tree example only if it relied solely on that retired local credential; it does not affect the published SDK or this example's source/API linkage.

Validation

  • env JAVA_HOME=/opt/homebrew/opt/openjdk@21 ./scripts/test
  • env JAVA_HOME=/opt/homebrew/opt/openjdk@21 ./scripts/lint
  • env JAVA_HOME=/opt/homebrew/opt/openjdk@21 ./scripts/build
  • clean :openai-java-example:compileJava
  • Gradle dependencyInsight and runtime graph review for Azure Identity, Azure Core HTTP Netty, Netty, Reactor Netty, Nimbus JOSE JWT, and json-smart
  • publication review confirming openai-java-example has no publishing tasks

@jbeckwith-oai jbeckwith-oai requested a review from a team as a code owner July 15, 2026 19:37
@openai-sdks

openai-sdks Bot commented Jul 15, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 13.105s for Java SDK PR #779.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 129ms
tests/chat-completions-create.test.ts ✅ Passed 386ms
tests/chat-completions-stream.test.ts ✅ Passed 259ms
tests/files-content-binary.test.ts ✅ Passed 268ms
tests/files-create-multipart.test.ts ✅ Passed 330ms
tests/files-list-pagination.test.ts ✅ Passed 218ms
tests/initialize-config.test.ts ✅ Passed 146ms
tests/instance-isolation.test.ts ✅ Passed 143ms
tests/models-list.test.ts ✅ Passed 275ms
tests/responses-background-lifecycle.test.ts ✅ Passed 230ms
tests/responses-body-method-errors.test.ts ✅ Passed 464ms
tests/responses-cancel-timeout.test.ts ✅ Passed 183ms
tests/responses-cancel.test.ts ✅ Passed 249ms
tests/responses-compact-retries.test.ts ✅ Passed 230ms
tests/responses-compact.test.ts ✅ Passed 245ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 216ms
tests/responses-create-advanced.test.ts ✅ Passed 865ms
tests/responses-create-disconnect.test.ts ✅ Passed 1.037s
tests/responses-create-errors.test.ts ✅ Passed 271ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 395ms
tests/responses-create-retries.test.ts ✅ Passed 179ms
tests/responses-create-stream-failures.test.ts ✅ Passed 164ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 205ms
tests/responses-create-stream-wire.test.ts ✅ Passed 4.906s
tests/responses-create-stream.test.ts ✅ Passed 71ms
tests/responses-create-terminal-states.test.ts ✅ Passed 203ms
tests/responses-create-timeout.test.ts ✅ Passed 205ms
tests/responses-create.test.ts ✅ Passed 231ms
tests/responses-delete.test.ts ✅ Passed 218ms
tests/responses-input-items-errors.test.ts ✅ Passed 277ms
tests/responses-input-items-list.test.ts ✅ Passed 286ms
tests/responses-input-items-options.test.ts ✅ Passed 253ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 183ms
tests/responses-input-tokens-count.test.ts ✅ Passed 284ms
tests/responses-malformed-inputs.test.ts ✅ Passed 3.892s
tests/responses-not-found-errors.test.ts ✅ Passed 274ms
tests/responses-parse.test.ts ✅ Passed 440ms
tests/responses-retrieve-retries.test.ts ✅ Passed 716ms
tests/responses-retrieve.test.ts ✅ Passed 208ms
tests/responses-stored-method-errors.test.ts ✅ Passed 930ms
tests/retry-behavior.test.ts ✅ Passed 3.524s
tests/sdk-error-shape.test.ts ✅ Passed 427ms

View OkTest run #29462769694

SDK merge (e96f9d194dfb) · head (a1985f83a66a) · base (845b2bc87fb0) · OkTest (fd7a761c81c1)

@jbeckwith-oai jbeckwith-oai force-pushed the codex/update-azure-identity branch from fe01171 to a1985f8 Compare July 16, 2026 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant